Skip to content

Conversation

@rgsl888prabhu
Copy link
Collaborator

This simplifies and update source build instructions.

@rgsl888prabhu rgsl888prabhu self-assigned this Jun 2, 2025
@rgsl888prabhu rgsl888prabhu added doc Improvements or additions to documentation non-breaking Introduces a non-breaking change labels Jun 2, 2025
@chris-maes
Copy link
Contributor

We say the following


**Note:** Using a conda environment is the easiest way to satisfy the library's dependencies.

- Create the conda development environment:
Please install conda if you don't have it already. You can install it from [https://docs.conda.io/en/latest/miniconda.html](https://docs.conda.io/en/latest/miniconda.html)

I think it might be good to

  1. Make a bit stronger statement that "is the easiest way to satisfy the library's dependencies". It sound like conda is really the only way to satisfy these dependencies. So maybe we should say
Building from source without conda is very difficult. We highly recommend that users build cuOpt inside a conda environment
  1. Include instructions for installing miniconda. I worry a bit about pointing users to miniconda installation instructions. Maybe we could add one or two lines that show how to install miniconda?

@rgsl888prabhu
Copy link
Collaborator Author

We say the following


**Note:** Using a conda environment is the easiest way to satisfy the library's dependencies.

- Create the conda development environment:
Please install conda if you don't have it already. You can install it from [https://docs.conda.io/en/latest/miniconda.html](https://docs.conda.io/en/latest/miniconda.html)

I think it might be good to

  1. Make a bit stronger statement that "is the easiest way to satisfy the library's dependencies". It sound like conda is really the only way to satisfy these dependencies. So maybe we should say
Building from source without conda is very difficult. We highly recommend that users build cuOpt inside a conda environment
  1. Include instructions for installing miniconda. I worry a bit about pointing users to miniconda installation instructions. Maybe we could add one or two lines that show how to install miniconda?

Providing instruction on miniconda would add additional burden on us to make sure it is in working condition, it would be better if they can just go through miniconda website to install. And they might also have many options. and we might not want to limit them to number of options.

@rgsl888prabhu
Copy link
Collaborator Author

We say the following


**Note:** Using a conda environment is the easiest way to satisfy the library's dependencies.

- Create the conda development environment:
Please install conda if you don't have it already. You can install it from [https://docs.conda.io/en/latest/miniconda.html](https://docs.conda.io/en/latest/miniconda.html)

I think it might be good to

  1. Make a bit stronger statement that "is the easiest way to satisfy the library's dependencies". It sound like conda is really the only way to satisfy these dependencies. So maybe we should say
Building from source without conda is very difficult. We highly recommend that users build cuOpt inside a conda environment
  1. Include instructions for installing miniconda. I worry a bit about pointing users to miniconda installation instructions. Maybe we could add one or two lines that show how to install miniconda?

Providing instruction on miniconda would add additional burden on us to make sure it is in working condition, it would be better if they can just go through miniconda website to install. And they might also have many options. and we might not want to limit them to number of options.

But let me update the link that will take them directly to linux installation page

@rgsl888prabhu
Copy link
Collaborator Author

@chris-maes @rg20 May I get your review ?

@rgsl888prabhu rgsl888prabhu merged commit ccbee4f into NVIDIA:branch-25.05 Jun 9, 2025
56 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Improvements or additions to documentation non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants